-
Notifications
You must be signed in to change notification settings - Fork 33
Add fps rate limiter #97
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: master
Are you sure you want to change the base?
Conversation
a810923 to
326b3f7
Compare
|
I'll have a go this weekend :) I'll also use this time to determine if changing the line order for 1080i helps my TV 🤔 |
|
Yep works great, having this in would be really nice! I used it like this which I think is correct? No FPS issues in levels now I have this on |
|
You need to use What you're doing causes 1 redundant 'pass', so removing it will make the game run a little smoother. |
|
Done. Though it did run fine without it |
|
@rickgaiser I think also adding for SD will be a good idea, but otherwise good to merge IMO |
|
Yes, I've already implemented that partially. Will try to finish when I have time. |
|
What I would like is a complete integration of hires/non-hires. Then instead of There's more functions from hires that can be removed in the same way. That would also simplify applications no longer needing the if/else in a few places for SD/HD. I tried to do this but it's taking me too much time so please review this PR as-is and leave the above as a note for those interested to tackle this issue. |
|
Makes sense. Though I think to make this non breaking couldn't this be a different function you call for handling on both modes? |
@freshollie can you try this?
Untested but should work. @fjtrujy what do you think? Can be added for non-HIRES as well.